Skip to content

Upgrade MCP server SDK to 2.0 - #2

Open
awantoch wants to merge 1 commit into
mainfrom
codex/mcp-sdk-v2
Open

Upgrade MCP server SDK to 2.0#2
awantoch wants to merge 1 commit into
mainfrom
codex/mcp-sdk-v2

Conversation

@awantoch

@awantoch awantoch commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the v1 umbrella SDK with @modelcontextprotocol/server 2.0 and use the official createMcpHandler HTTP entry point
  • move bearer validation onto SDK requireBearerAuth while preserving Supabase token scoping and RFC 9728 protected-resource metadata
  • migrate all tool inputs to complete Standard Schema objects and remove redundant handler-side validation
  • add the v2 client as a dev-only E2E harness and verify authenticated protocol negotiation, discovery, and team_list
  • declare the Node 20.9 minimum shared by Next.js and MCP v2

Compatibility notes

  • modern 2026-07-28 traffic uses JSON response mode
  • 2025-era traffic remains available through the SDK stateless fallback and is covered by the existing server suite
  • CORS now permits the modern MCP protocol headers

Verification

  • npm run typecheck
  • npm test — 92 files, 411 tests
  • npm run lint
  • npm run build
  • npm run test:e2e — 15 tests

References

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-01T04:09:17.663496Z 0ddd39d PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant